-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Search category autocomplete includes group category #2731
Search category autocomplete includes group category #2731
Conversation
✅ Deploy Preview for actualbudget ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Bundle Stats — desktop-clientHey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle. As this PR is updated, I'll keep you updated on how the bundle size is impacted. Total
Changeset
View detailed bundle breakdownAdded No assets were added Removed No assets were removed Bigger
Smaller No assets were smaller Unchanged
|
Bundle Stats — loot-coreHey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle. As this PR is updated, I'll keep you updated on how the bundle size is impacted. Total
Changeset No files were changed View detailed bundle breakdownAdded No assets were added Removed No assets were removed Bigger No assets were bigger Smaller No assets were smaller Unchanged
|
0a56ddc
to
088e06e
Compare
We generally try to avoid having extra settings like this. Thats partly why the discussion has so heated around this topic. Its either one way or another, or we have to add an extra setting. Undesirable no matter how we shake it. A thought I had just now. Having settings be contextual is nicer than global settings. Could we put this setting inside the autocomplete somehow instead of the settings page? |
Another thought. Could we have a way to tell the autocomplete to include groups incidentally? Like type "$bills" to search for the group bills? That is still getting to magic syntax which is its own kind of non-optimal |
@youngcw I agree that having a setting like that is not optimal. I'm not sure on how to add the setting in the dropdown in a nice and unobtrusive way. Adding it at the bottom of the dropdown would not be great: once set to the user preference, it will be rather useless and keep displaying all the time. I can add a magic syntax instead, but its discoverability will be rather poor, and it will not feel really natural (especially on mobile) I can also move the new setting in the "advanced settings", would it be better? |
088e06e
to
f3fa7a6
Compare
thats a good point that mobile would probably be a pain with magic syntax |
- Added a new option switch to have the category autocomplete match on category group names
f3fa7a6
to
6c08db7
Compare
@youngcw I moved the new setting in the "advanced settings" sections. I feel it's the best that can be done to support the two ways of using the category autocomplete. |
We've been back-and-forth with this for a while now and the current consensus is that category groups should not be included in search results. We also do not want to have a setting for each and every little UI quirk as it would unnecessarily clutter the product. Instead - we have opinionated workflows. I can see two options how to proceed here:
edit: personally - I don't care about including/excluding category groups here. What I do care about is the consistency in the product. We can't add a feature, then remove it, then add it, then again remove it. |
@MatissJanis Ok, duly noted then. A few points before leaving:
|
Resolve #2324
Reading the discussion on #1681 and #1767, I understand that people either want the category autocomplete to consider group names when searching or not. (I happen to be in the group wishing for it to match group names 😄)
I added a new option switch to have the category autocomplete match on category group names.
It can be tested on the test budget, the option is in the application settings and using the "Bills" category group name in the category autocomplete demonstrates it.